wl-clipboard-rs only works with compositors using the wlr-data-control protocol, it doesn't work on GNOME's mutter for instance:

That is fine as anything apart from wlroots is out of our project scope.

Reading up on the protocol, it appears that a process can only offer data to be copied only for as long as it stays alive. This is mostly fine for GUI apps but problems arise when you want to copy something to the clipboard and exit as a terminal app. Basically with this PR wayshot will offer up the screenshot for a small moment and immediately exit.

This is a very well known limitation of the protocol and partly the reason why it was left out of the feature set for a while.

The way to circumvent this is to keep the process alive. wl-clipboard-rs's version of wl-copy for instance does some unsafe shenanigans to fork itself, disconnect stdin/stdout and maintains the image in-memory till some other process overwrites the clipboard: https://github.com/YaLTeR/wl-clipboard-rs/blob/10b35fb2699a0ff65888b1220804bb0c44b65e0f/wl-clipboard-rs-tools/src/bin/wl-copy.rs#L160 I believe this is the same thing wl-copy does. We'll also have to do something similar to achieve parity with the wayshot --stdout | wl-copy method.

Maybe we can explore this approach if it can be robust enough. I say this from a preliminary overview and naturally I'd need some more code review before I can confirm this.

Before you write any further code. I'd like to kindly request you to rebase your work on freeze-feat-andreas branch as we're currently working on some breaking changes for libwayshot and wayshot.

I think It'll probably be easier to close this PR and do another one on that branch with how changed it is, I'll admit I don't know enough about git convention to be sure of this tho so advice will be appreciated.

I'm also considering having the clipboard copy feature not be exclusive of the stdout or file save modes as it is in this PR. That would also make it a feature not trivially synonymous to wayshot - | wl-copy. wayshot can copy to the clipboard and simultaneously save to a file/stdout.

@CheerfulPianissimo Apologies for the late reply, if you're not comfortable with rebase then feel free to open a new pr with the target branch set to the devel branch.